Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] Added Toggle to magnetPickupSystem / [FIX] Ore Processor Fix #15

Merged
merged 12 commits into from
Sep 3, 2024

Conversation

HellCatten
Copy link
Contributor

[NEW] MagnetPickupSystem ItemToggle added / [FIX] Ore processor fixed

Описание.

[NEW] Добавлен toggle / переключатель для ore bag / сумки для руды. Изначально выключен.

[FIX] Исправлена ошибка в компонентах ore processor / плавитель руды. Для плавителя есть специальный компонент/система MaterialStorageMagnetPickupSystem. Позволяет плавителю засасывать руду в тайле. Однако из-за отсутствия PlaceableSurfaceComponent сгрузить руду из сумки невозможно. Только в ручную.

Добавление данного компонента исправляет ситуацию.


Медиа

[FIX]
https://github.com/user-attachments/assets/d82f8aeb-4696-4ec1-a8f0-62f03be9a12e

[NEW]

2024-08-23.22-22-24.mp4
Список

Example Media Embed


Изменения

🆑 Hell_Cat

  • add: added toggle in ore bag
  • fix: Ore processor fixed

@riddleridou riddleridou changed the title [NEW] Added toggle to magnetPickupSystem / [FIX] ore processor fix [NEW] Added Toggle to magnetPickupSystem / [FIX] Ore Processor Fix Aug 23, 2024
@HellCatten
Copy link
Contributor Author

Можно добавить разные приколы к этой фичи.

Как эффект состояния вкл. Изменение спрайта / добавление эффектов.

Copy link
Member

@Spatison Spatison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

К строчкам кода изменённым не в папках вайта следует добавлять комментарий.

Одна строчка кода // WD EDIT
// WD EDIT START
Три и более строчки 
// WD EDIT END
В прототипе # WD EDIT

@HellCatten HellCatten changed the title [NEW] Added Toggle to magnetPickupSystem / [FIX] Ore Processor Fix [Feature] Added Toggle to magnetPickupSystem / [FIX] Ore Processor Fix Aug 24, 2024
@HellCatten
Copy link
Contributor Author

К строчкам кода изменённым не в папках вайта следует добавлять комментарий.

Одна строчка кода // WD EDIT
// WD EDIT START
Три и более строчки 
// WD EDIT END
В прототипе # WD EDIT

Done

@HellCatten
Copy link
Contributor Author

Work in process

@HellCatten
Copy link
Contributor Author

Не могу сделать по заказу Remuchi, чтобы оно работало в руках. Придется лезть в HandsSharedSystem и перелопачивать слишком много чего.

Либо убирать строки ниже. Тогда ore_bag будет магнитить вне инвентаря, в руках и лежа на карте.

if (!_inventory.TryGetContainingSlot((uid, xform, meta), out var slotDef)) continue; if ((slotDef.SlotFlags & comp.SlotFlags) == 0x0) continue;

Либо делать if на разрешение подбирать лежа на карте. Проверка на слот по факту не нужна, потому что ore_bag в инвентаре может лежать только в слоте BELT
if (comp.MapPickup || !_inventory.TryGetContainingSlot((uid, xform, meta), out var slotDef)) continue;

На данный момент жду спрайта для ore_bag_on

@Remuchi Remuchi marked this pull request as draft August 27, 2024 10:24
@Remuchi
Copy link
Collaborator

Remuchi commented Aug 27, 2024

Как только закончишь работу - переоткроешь

Copy link
Contributor

github-actions bot commented Aug 27, 2024

RSI Diff Bot; head commit 9a1e710 merging into 2ea1619
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Specific/Mining/ore_bag.rsi

State Old New Status
orebag_on Added

Resources/Textures/Objects/Specific/Mining/ore_bag_holding.rsi

State Old New Status
orebag_on Added

Edit: diff updated after 9a1e710

@HellCatten
Copy link
Contributor Author

Добавлены спрайты

@HellCatten
Copy link
Contributor Author

image

@HellCatten HellCatten marked this pull request as ready for review August 27, 2024 10:48
@Remuchi
Copy link
Collaborator

Remuchi commented Aug 27, 2024

Ошибки в линтинге

@HellCatten
Copy link
Contributor Author

Failed SpawnAndDeleteAllEntitiesInTheSameSpot [19 s]

Не знаю в чем может быть проблема

@Spatison
Copy link
Member

Failed SpawnAndDeleteAllEntitiesInTheSameSpot [19 s]

Не знаю в чем может быть проблема

Ошибка не связана с твоим ПРом. При повторной запуске чеков скорее всего она сама исправится.

Copy link
Collaborator

@Remuchi Remuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так же запушь свои изменения, связанные с работой его вне пояса. Пускай работает и на земле, ничего страшного.

@HellCatten
Copy link
Contributor Author

Удалил ClotDef check из-за ненадобности. Ore_bag в инвентаре может быть только в 1 слоте - Belt.

В MagnetPickupSystem Добавлен DataField ForcePickup. При помощи него регулируется работа системы.

True: Магнитит все. В руках, на земле.
False: Работает как раньше. Магнитит только на Belt

Copy link
Collaborator

@Remuchi Remuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ForcePickup не предиктед. Из-за чего при отключении происходит вечная попытка поднять руду.

Content.Shared/Storage/Components/MagnetPickupComponent.cs Outdated Show resolved Hide resolved
@HellCatten HellCatten requested a review from Remuchi September 2, 2024 09:45
@HellCatten
Copy link
Contributor Author

Исправил Predicted через NetworkedComponent и AutoNetworkedField

@Remuchi Remuchi merged commit 86534e1 into WWhiteDreamProject:master Sep 3, 2024
9 of 10 checks passed
riddleridou added a commit that referenced this pull request Sep 3, 2024
@HellCatten HellCatten deleted the Ore-bag-toggle branch September 6, 2024 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants